Really good explanation of scope and variable declarations here, just wanted to highlight the extra if statement you added. Are you sure it handles the empty string case like you describe?

this is the best explanation i could come up with.

By checking typeof str !== "string", you are asking: "If the input is anything other than a string, stop processing and just return the input exactly as it is." If you send a string: It skips the if block and proceeds to do the work.
If you send a number: The if condition is true, so it returns the number and ignores the rest of the function.

So your understanding of what the if statement does (as you've explained here) is good, but that's not what you've said in the comments 🙂 You said:

the if condition i added is to ensure that if an empty string is passed to the function, it will return the empty string instead of throwing an error.

But as you have described, the if condition checks for the data type, not whether the string is empty. How would you check for that?

yes, the initial statement was not correct, i looked at it critically a saw that this if statement will trow an error, because i did not add this ( | | ) operator, Because of the | | operator, if the string is empty, the function returns str. to handle it i will add || str.length === 0) to the if condition. and this will handle it.

Please add this second condition then 🙂

Almost there Ogbemi - just make sure to add this condition and then I'm happy to mark as complete!